-
Notifications
You must be signed in to change notification settings - Fork 695
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New module: custom/splitfastqbylane #2837
base: master
Are you sure you want to change the base?
Conversation
Is this module still useful? |
In my and my colleague's opinion yes. We still have previously merged fastq files that come our way, and for base quality recalibration purposes, we align them separately so that each read gets a read-group label based on the flowcell/lane it came from. i asked for feedback on this module some time ago. |
Yes, I see that nobody reviewed the module 😔. |
BQSR will read the lane tag in the BAM file, assuming you're using it correctly. Other than that the code looks fine but we should update to nf-test now. |
So shall we swap this to nf-test and merge it in? |
This is a custom module using awk to split a single fastq or fastq pair into multiple fastqs or fastq pairs, where each comes from a single lane+flowcell source. This is for when the raw data input is merged. This tool uses an awk statement to read the input file and direct the output to different files based on the content of the fastq header.
PR checklist
Closes #2836
versions.yml
file.label
PROFILE=docker pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
PROFILE=singularity pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
PROFILE=conda pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware